website: Add Kubeflow AI Policy#4336
Conversation
Signed-off-by: Saad Zaher <szaher@redhat.com>
|
@kubeflow/kubeflow-steering-committee @kubeflow/kubeflow-notebooks-security @kubeflow/kubeflow-hub-team @kubeflow/kubeflow-kale-team @kubeflow/wg-training-leads @kubeflow/wg-notebooks-leads @kubeflow/wg-automl-leads |
Signed-off-by: Saad Zaher <szaher@redhat.com>
|
I have removed the original one and replaced it with a short message for the website. please take a look |
Co-authored-by: Julius von Kohout <45896133+juliusvonkohout@users.noreply.github.com> Signed-off-by: Saad Zaher <szaher@redhat.com>
content/en/docs/about/ai_policy.md
Outdated
| aliases = ["/docs/ai-policy/"] | ||
| +++ | ||
|
|
||
| AI tools may be used to draft or review code/documentation, but the human contributor remains fully responsible for every change: you must understand it, verify it (build/lint/test), and be able to explain it during review, your PR may be closed if you cannot |
There was a problem hiding this comment.
As we discussed on the call, @szaher shall we add bullet points from your PR here: kubeflow/community#943?
Also, I suggest to tell that users should review contributing guide + AGENTS.md if available.
christian-heusel
left a comment
There was a problem hiding this comment.
Sounds good with the minor additions that I have posted in my review comments.
Maybe we should also add references to this document from other parts in the website?
content/en/docs/about/ai_policy.md
Outdated
| aliases = ["/docs/ai-policy/"] | ||
| +++ | ||
|
|
||
| AI tools may be used to draft or review code/documentation, but the human contributor remains fully responsible for every change: you must understand it, verify it (build/lint/test), and be able to explain it during review, your PR may be closed if you cannot |
There was a problem hiding this comment.
We're switching the POV here ("the human contributor" vs "you), that should probably be aligned. I think just adding "but you as the human contributor" might already be sufficient 😄
content/en/docs/about/ai_policy.md
Outdated
| aliases = ["/docs/ai-policy/"] | ||
| +++ | ||
|
|
||
| AI tools may be used to draft or review code/documentation, but the human contributor remains fully responsible for every change: you must understand it, verify it (build/lint/test), and be able to explain it during review, your PR may be closed if you cannot |
There was a problem hiding this comment.
The sentence you end with a "."
content/en/docs/about/ai_policy.md
Outdated
|
|
||
| Do not use AI to author responses in review threads or community discussions, reviewers expect to engage with you directly. | ||
|
|
||
| All contributions must still meet Kubeflow’s quality and security standards and comply with DCO, Apache-2.0, and third-party IP/licensing requirements. No newline at end of file |
There was a problem hiding this comment.
| All contributions must still meet Kubeflow’s quality and security standards and comply with DCO, Apache-2.0, and third-party IP/licensing requirements. | |
| All contributions must still meet Kubeflow’s quality and security standards and comply with [DCO](https://developercertificate.org/), the [Apache-2.0](https://github.com/kubeflow/kubeflow/blob/master/LICENSE) license, and third-party IP/licensing requirements. |
|
@szaher Did you get a chance to add bullet points, so we can move forward? |
Signed-off-by: Saad Zaher <szaher@redhat.com>
Signed-off-by: Saad Zaher <szaher@redhat.com>
|
@andreyvelich can you please review it again? |
andreyvelich
left a comment
There was a problem hiding this comment.
Thanks @szaher, just a small suggestion!
/lgtm
/assign @franciscojavierarceo @thesuperzapper @juliusvonkohout @chasecadet
content/en/docs/about/ai_policy.md
Outdated
| aliases = ["/docs/ai-policy/"] | ||
| +++ | ||
|
|
||
| Please review the Kubeflow [contributing](contributing.md) guide first. |
There was a problem hiding this comment.
We should also specify that if AGENTS.md files are present in Kubeflow sub-projects, agents should be configured to read them for additional context.
There was a problem hiding this comment.
If I remember @thesuperzapper mentioned this is not a policy but more of contributing guide or suggestion
There was a problem hiding this comment.
Yes agents.md and copilot-instructions.md
There was a problem hiding this comment.
@szaher Please can you add this, so we can merge this PR.
You can say that if those files are available, please review them for additional context.
Signed-off-by: Francisco Javier Arceo <farceo@redhat.com>
franciscojavierarceo
left a comment
There was a problem hiding this comment.
/lgtm
/approve
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: franciscojavierarceo The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Description of Changes
As discussed during the community meeting, I am raising this PR to introduce the kubeflow AI Policy.
Related Issues
Closes: #
Related: #
Checklist